-
Notifications
You must be signed in to change notification settings - Fork 519
use PSGet v3 #2932
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use PSGet v3 #2932
Conversation
Ah I see, it gives a warning, nice: but also :( for the error. @JamesWTruher? |
I like to run |
@alerickson Good to know! |
I guess in this context it might make more sense to run ((get-command find-module).module).Version |
This is an odd error to me. Why is it spelled "Powershell" in the path? |
Okay, just updating with convo Tyler and I had offline: beta9 is the latest psget release that handles some unix path changes, there is however, a bug with the -debug param in Install/Save (I realize it's not being used right now, just a warning) |
.vsts-ci/templates/ci-general.yml
Outdated
- pwsh: | | ||
Install-Module -Scope CurrentUser -Force -AllowPrerelease PowerShellGet | ||
Install-Module -Scope CurrentUser -Force -AllowPrerelease CompatPowerShellGet | ||
Install-Module -Scope CurrentUser -Force -AllowPrerelease -AllowClobber PowerShellGet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't need -AllowClobber for PSGet since it's entirely new cmdlet names
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It'll scale better once PSGet v3 ships with PowerShell.
ok I'm gonna stop here for now. Looks like we have some more work:
|
Surely that's a bug, right? I know, don't call you Shirley. |
@rkeithhill personally I agree with you. I think PSGet v3 should register PSGallery by default. I only did it in the commit to see what would happen if it was already registered. |
This moves us to use the latest preview of PowerShellGet v3 and the compat PSGet module